-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
docs(svelteKit): Update build-time options for source maps #15283
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Bundle ReportChanges will decrease total bundle size by 15 bytes (-0.0%) ⬇️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-server-cjsAssets Changed:
view changes for bundle: sentry-docs-client-array-pushAssets Changed:
|
docs/platforms/javascript/guides/sveltekit/configuration/build/index.mdx
Outdated
Show resolved
Hide resolved
| <SdkOption name="sentryUrl" type="string" envVar="SENTRY_URL" defaultValue="https://sentry.io/"> | ||
|
|
||
| The base URL of your Sentry instance. Only relevant if you're using a self-hosted or Sentry instance other than sentry.io. | ||
| The base URL of your Sentry instance. Only relevant if you're using a self-hosted or Sentry instance other than sentry.io. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a case where someone is using a different Sentry instance that isn't self-hosted? Or can this be simplified to "Only relevant if you're using a self-hosted Sentry instance."?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, but I would still include the first sentence as it explains what it is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎃
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
…s-examples * origin/master: (478 commits) docs(godot): Structured logging (#15287) fix(batch-processor): Don't enforce starting timer (#15309) feat(node): Add new options to nativeNodeFetchIntegration (#15282) Updated dotnet options to include all defaults (#15218) ref: add docs that logs channel is optional (#15172) docs(svelteKit): Update build-time options for source maps (#15283) fix(python): Fix paragraph order in Strawberry docs (#15301) Added docs explaining how to disable Sentry Native in the .NET SDK (#15293) Updated Unity options with missing defaults (#15219) docs(otlp): Update docs for client SDK w3c traceparent support (#15294) Adding defaults for Godot options (#15216) feat(crons): Document broken monitors feature (#15280) fix: Remove OTLP logs from EA list (#15288) Bump API schema to 06047b60 (#15276) ref(nextjs): Move manual instrumentation examples for ai into Edge section (#15284) docs(otlp): change status to open beta (#15285) Hono runtime and middleware doc updates (#14853) feat(logs): Clarify API for Hub/Scope and Client (#15281) docs(astro/nuxt): Remove `sourceMapsUploadOptions` option nesting (#14905) docs(profiling): Add docs for Profiling API (#15258) ...
DESCRIBE YOUR PR
Updating docs based on this PR: getsentry/sentry-javascript#17413
Related to this project: getsentry/sentry-javascript#14212
IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes: